Skip to content

tests/functional: Reduce max-call-depth for stack overflow tests#14920

Merged
xokdvium merged 1 commit intomasterfrom
fix-ci-oom
Jan 6, 2026
Merged

tests/functional: Reduce max-call-depth for stack overflow tests#14920
xokdvium merged 1 commit intomasterfrom
fix-ci-oom

Conversation

@xokdvium
Copy link
Contributor

@xokdvium xokdvium commented Jan 5, 2026

Motivation

This OOMs VM tests in CI and it's just wasteful.

Context


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

This OOMs VM tests in CI and it's just wasteful.
@xokdvium xokdvium requested a review from roberth January 5, 2026 23:37
@xokdvium xokdvium requested a review from edolstra as a code owner January 5, 2026 23:37
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Jan 5, 2026
@xokdvium xokdvium added this pull request to the merge queue Jan 6, 2026
Merged via the queue into master with commit 22e46fb Jan 6, 2026
18 checks passed
@xokdvium xokdvium deleted the fix-ci-oom branch January 6, 2026 03:04
@roberth
Copy link
Member

roberth commented Jan 6, 2026

I'm surprised that 10000 non returned calls would OOM, but lgtm.

EDIT: context

@xokdvium
Copy link
Contributor Author

xokdvium commented Jan 6, 2026

Partly due to the vm getting only 1GB of RAM :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

with-tests Issues related to testing. PRs with tests have some priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants